Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SDK-2095] Added new unit tests for main classes #1302

Merged
merged 21 commits into from
Oct 25, 2023
Merged

[SDK-2095] Added new unit tests for main classes #1302

merged 21 commits into from
Oct 25, 2023

Conversation

nsingh-branch
Copy link
Contributor

Reference

SDK-2095 -- iOS Test coverage - Add tests for Branch.m, BranchEvent, BranchUniversalObject, and BranchShareLink

Summary

Add more tests to increase coverage, mostly in the main Branch.m class.

Motivation

To increase test coverage percentage for main public methods.

Type Of Change

  • Bug fix (non-breaking change which fixes an issue)

Testing Instructions

Run the new tests and ensure they are relevant and in the right locations.

cc @BranchMetrics/saas-sdk-devs for visibility.

@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Merging #1302 (dae58df) into master (590a11c) will increase coverage by 1.99%.
Report is 19 commits behind head on master.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1302      +/-   ##
==========================================
+ Coverage   50.67%   52.66%   +1.99%     
==========================================
  Files          68       68              
  Lines       10106    10109       +3     
  Branches     3889     3892       +3     
==========================================
+ Hits         5121     5324     +203     
+ Misses       4742     4524     -218     
- Partials      243      261      +18     
Files Coverage Δ
BranchSDK/BranchActivityItemProvider.m 0.00% <ø> (ø)
BranchSDK/Branch.m 41.96% <0.00%> (+4.36%) ⬆️

... and 12 files with indirect coverage changes

@echo-branch echo-branch merged commit f99239a into master Oct 25, 2023
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants